Skip to content

Refactor(durabletask): migrate to unified folder structure#44308

Merged
qiaozha merged 3 commits into
Azure:mainfrom
v-xuto:durabletask-refactor
Jul 9, 2026
Merged

Refactor(durabletask): migrate to unified folder structure#44308
qiaozha merged 3 commits into
Azure:mainfrom
v-xuto:durabletask-refactor

Conversation

@v-xuto

@v-xuto v-xuto commented Jun 30, 2026

Copy link
Copy Markdown
Member

This PR migrates the durabletask specification to the new unified folder structure according to Azure folder structure guidelines.

Change:

  • Consolidated TypeSpec and Swagger files under resource-manager\Microsoft.DurableTask\DurableTask
  • Updated tspconfig.yaml with correct paths and ARM types directory
  • Fixed all common-types references and input-file paths in readme.md
  • Verified TypeSpec compilation works correctly
  • Updated the relevant paths in specification/suppressions.yaml file
  • Add rpaas readme.md under resource-manger

Benefits:

  • Unified structure for better maintainability
  • Aligns with Azure folder structure standards
  • Maintains backward compatibility
  • Supports future TypeSpec migration efforts

@v-xuto v-xuto added the FolderMigrationV2 A specific label for PRs like folder structure migration into v2. label Jun 30, 2026
@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR is in purview of the ARM review (label: ARMReview). This PR must get ARMSignedOff label from an ARM reviewer.
    This PR is not ready for ARM review (label: NotReadyForARMReview). This PR will not be reviewed by ARM until relevant problems are fixed. Consult the rest of this Next Steps to Merge comment for details.
    Once the blocking problems are addressed, add to the PR a comment with contents /azp run. Automation will re-evaluate this PR and if everything looks good, it will add WaitForARMFeedback label which will put this PR on the ARM review queue.
    For details of the ARM review, see aka.ms/azsdk/pr-arm-review
  • ❌ This PR is in purview of the ARM review (label: ARMReview). This PR must get ARMSignedOff label from an ARM reviewer.
    This PR has ARMModelingReviewRequired label. This means it is introducing a new Resource Provider namespace or a new resource type. New RPs and new resource types require a discussion with the ARM Modeling Review team before merging.

    If you haven't discussed yet:
    Please schedule a meeting at ARM API Modeling Office Hours.

    If ARMModelingSignedOff label was removed by automation:
    This happens when the lease file is not found. Please check with the PM you discussed your design with to confirm if the lease file PR has been merged. In the public repo, lease files reflect immediately after merge. In the private repo (azure-rest-api-specs-pr), lease files typically sync within a few hours, but it can take up to 1-2 business days depending on sync pipeline timing and queue load. Please wait for the sync to complete.
    For details of the ARM review, see aka.ms/azsdk/pr-arm-review


Comment generated by summarize-checks workflow run.

@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Language API Review for Package
Swagger Microsoft.DurableTask-DurableTask
TypeSpec Microsoft.DurableTask
Go sdk/resourcemanager/durabletask/armdurabletask
JavaScript @azure/arm-durabletask
Python azure-mgmt-durabletask
C# Azure.ResourceManager.DurableTask
Java com.azure.resourcemanager:azure-resourcemanager-durabletask

Comment generated by After APIView workflow run.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Namespace Review Required

Plane: Management Plane

Language Proposed Namespace Format Status Approvers
python azure.mgmt.durabletask ⚠️ Invalid ⏳ Pending @ArthurMa1978, @m-nash
java (package) com.azure.resourcemanager.durabletask ⏳ Pending @ArthurMa1978, @m-nash
java (artifact) azure-resourcemanager-durabletask ⏳ Pending @ArthurMa1978, @m-nash
go github.com/Azure/azure-sdk-for-go/{service-dir}/armdurabletask ⚠️ Invalid ⏳ Pending @ArthurMa1978, @m-nash
dotnet Azure.ResourceManager.DurableTask ⏳ Pending @ArthurMa1978, @m-nash
typescript @azure/arm-durabletask ⏳ Pending @ArthurMa1978, @m-nash

⚠️ Format issues detected:

  • python: "azure.mgmt.durabletask" does not match expected format: azure-mgmt-{name} (lowercase)
  • go: "github.com/Azure/azure-sdk-for-go/{service-dir}/armdurabletask" does not match expected format: sdk/resourcemanager/{name}/arm{name} (lowercase)

Format validation does not block approval but should be reviewed.

How to approve:

  • Per language: apply namespace-<language>-approved label
  • All at once: apply namespace-approved-all label (shortcut for mgmt plane)

Merge is blocked until all languages are approved.

Approver list: .github/namespace-approvers.yml
Process: .github/workflows/src/namespace-approval/NAMESPACE-REVIEW-PROCESS.md
Namespaces extracted from tspconfig.yaml emitter options

@qiaozha qiaozha added the PublishToCustomers Acknowledgement the changes will be published to Azure customers. label Jul 9, 2026
@qiaozha qiaozha merged commit 9c760a1 into Azure:main Jul 9, 2026
66 of 70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants